Skip to content

fix: patch transitive dep CVEs (same-major overrides)#49

Closed
forbiddenlink wants to merge 1 commit into
mainfrom
fix/dep-vulns
Closed

fix: patch transitive dep CVEs (same-major overrides)#49
forbiddenlink wants to merge 1 commit into
mainfrom
fix/dep-vulns

Conversation

@forbiddenlink

Copy link
Copy Markdown
Owner

pnpm overrides pinning vulnerable transitive deps to nearest patched same-major version. trivy HIGH/CRIT reduced. No major bumps. Auto-generated security sweep.

Adds pnpm overrides for vulnerable transitive deps (esbuild, grpc-js,
etc.) to their nearest patched same-major release. Cleared/reduced trivy
HIGH/CRIT findings; no major bumps.
Copilot AI review requested due to automatic review settings June 14, 2026 02:00
@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
skill-mapper Ignored Ignored Preview Jun 14, 2026 2:00am

Request Review

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@coderabbitai

coderabbitai Bot commented Jun 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@forbiddenlink, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 59 minutes and 15 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5bcedb64-6594-43f3-bed3-ed6c8d6eb822

📥 Commits

Reviewing files that changed from the base of the PR and between 4d221d3 and 86d58ed.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/dep-vulns

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates pnpm overrides (and the generated lockfile) to mitigate CVEs in transitive dependencies by forcing resolution to patched versions without introducing major-version upgrades.

Changes:

  • Tighten the esbuild override to ^0.28.1 and propagate the resolved version across the dependency graph.
  • Add an override for fast-uri to ^3.1.2 (pulled in via ajv) and update the lockfile accordingly.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updates pnpm.overrides to enforce patched versions for esbuild and fast-uri.
pnpm-lock.yaml Regenerates lockfile entries to reflect the new overrides and resolved versions.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@debugg-ai

debugg-ai Bot commented Jun 14, 2026

Copy link
Copy Markdown

🧪 E2E Test Results

📝 Commit: 86d58ed5 (PR #49)

📊 Summary

Status: ❌ FAILED

Pass Rate: 25.0% (1/4)
Duration: 2.2 minutes
Breakdown: ✅ 1 passed | ❌ 3 failed

📋 Test Results

Status Test Name Category Duration Recording Final Screenshot
✅ Pass User can view Skill Dependency Mapper interface el... general 7.7s 🎬 View View
❌ Fail User can see help or instructions on Node-Based UI... general 41.5s 🎬 View N/A
❌ Fail User can see help or instructions on Skill Depende... general 75.1s 🎬 View View
❌ Fail User can load Node-Based UIs in React page general 7.8s 🎬 View View

Generated by Debugg AI 🤖

@forbiddenlink

Copy link
Copy Markdown
Owner Author

Superseded: CVE overrides re-applied directly on current main.

@forbiddenlink forbiddenlink deleted the fix/dep-vulns branch June 17, 2026 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants